Skip to content

Conversation

@stuartleeks
Copy link
Contributor

No description provided.

@russcam
Copy link
Contributor

russcam commented Oct 16, 2018

Thanks @stuartleeks, that's a terrible typo 😭 Would you mind signing the CLA so we can pull it in?

@stuartleeks
Copy link
Contributor Author

done :-)

@russcam
Copy link
Contributor

russcam commented Oct 17, 2018

Thanks! Sorry to be a pain, would you also be able to update it in https://github.com/elastic/elasticsearch-net/blob/6.x/src/Tests/Tests/ClientConcepts/LowLevel/GettingStarted.doc.cs#L67 ? The asciidoc file is generated from that source, so will only end up being overwritten on the next doc generation!

@stuartleeks
Copy link
Contributor Author

I'm getting a 404 on that path...

@russcam
Copy link
Contributor

russcam commented Oct 17, 2018

Apologies, the test projects have moved around a bit and the docs not yet regenerated. I've updated the link in the previous comment.

@russcam russcam mentioned this pull request Oct 17, 2018
@stuartleeks
Copy link
Contributor Author

@russcam
Copy link
Contributor

russcam commented Oct 17, 2018

@stuartleeks
Copy link
Contributor Author

bingo - done!

Copy link
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@russcam russcam merged commit b2bac81 into elastic:6.x Oct 17, 2018
@russcam
Copy link
Contributor

russcam commented Oct 17, 2018

Thanks @stuartleeks!

russcam pushed a commit that referenced this pull request Oct 17, 2018

(cherry picked from commit b2bac81)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants